Skip to content

FIX ping DecodeV4Error #23 #15#45

Merged
aisk merged 3 commits into
aisk:masterfrom
jcaude:master
Dec 4, 2025
Merged

FIX ping DecodeV4Error #23 #15#45
aisk merged 3 commits into
aisk:masterfrom
jcaude:master

Conversation

@jcaude

@jcaude jcaude commented Dec 1, 2025

Copy link
Copy Markdown
Contributor

Hi,

I propose a solution to fix the issue some of us are facing with your package, in particular on ubuntu. I change two parts:

  1. I skip the IPv4 decoding, because we get the ICMP packet (suggested by @lz520520)
  2. I compare the payload because the ident is modified for whatever reason

Things are now ok on my side (tested on various hosts), I hope it would help.

Sincerely,
jC

Comment thread src/ping.rs
@aisk aisk merged commit 9c03e1e into aisk:master Dec 4, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants